Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hack/scripts-dev: add "docker-dns-example-certs-common-name-run" #8961

Merged
merged 3 commits into from
Dec 15, 2017

Conversation

gyuho
Copy link
Contributor

@gyuho gyuho commented Dec 1, 2017

Address #8502.
Separate out from #8954.

/cc @fanminshi

--endpoints=https://m1.etcd.local:2379,https://m2.etcd.local:22379,https://m3.etcd.local:32379 \
get abc

sleep 1s && printf "\n"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no sleep?
same as many places down below/

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's keep sleep here. I am trying to put example workflow here with print statements. Otherwise, it's hard to follow.

get abc

sleep 1s && printf "\n"
echo "Step 1. creating root role"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't quite understand this test.
why are we creating root user and root role?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

User must create root user and role first. It's special user and role. https://github.com/coreos/etcd/blob/master/Documentation/op-guide/authentication.md#special-users-and-roles

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes.

@gyuho gyuho force-pushed the test-scripts-4 branch 6 times, most recently from cf95d67 to 5071711 Compare December 7, 2017 15:26
@codecov-io
Copy link

codecov-io commented Dec 7, 2017

Codecov Report

❗ No coverage uploaded for pull request base (master@014c375). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #8961   +/-   ##
=========================================
  Coverage          ?   76.06%           
=========================================
  Files             ?      359           
  Lines             ?    29837           
  Branches          ?        0           
=========================================
  Hits              ?    22695           
  Misses            ?     5570           
  Partials          ?     1572

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 014c375...c0afdbe. Read the comment docs.

Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
mv ca.pem ca.crt
openssl x509 -in ca.crt -noout -text

# generate wildcard certificates DNS: *.etcd.local
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it doesn't seems like you are generating wildcard certificates from server-ca-csr.json

"hosts": [
     "m1.etcd.local",
     "m2.etcd.local",
     "m3.etcd.local",
     "127.0.0.1",
     "localhost"
   ]

@fanminshi
Copy link
Member

this script is more a demo of how to use common name than a test script. suggest rename this as a demo script.

Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho gyuho changed the title hack/scripts-dev: add "docker-dns-test-certs-common-name-run" hack/scripts-dev: add "docker-dns-example-certs-common-name-run" Dec 15, 2017
@gyuho gyuho merged commit 91415f8 into etcd-io:master Dec 15, 2017
@gyuho gyuho deleted the test-scripts-4 branch December 15, 2017 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants